Skip to content

Conversation

@fangchenli
Copy link

@fangchenli fangchenli commented Jan 30, 2026

Summary

  • Remove Scala 2.12 from supported versions
  • Remove scala-2.12- version-specific source directories
  • Update build.sbt to only support Scala 2.13 and 3.3.4
  • Update CI workflow to remove 2.12 from test matrix
  • Use Scala 2.13+ APIs (IterableOnce, jdk.CollectionConverters)

Breaking Change

BREAKING: Scala 2.12 is no longer supported. Users must upgrade to Scala 2.13 or 3.x.

Test plan

  • All Scala 2.13 tests pass locally
  • CI passes for Scala 2.13 and 3.3.4

Note: Scala 3 test compilation has pre-existing issues unrelated to this change.

🤖 Generated with Claude Code

fangchenli and others added 3 commits January 30, 2026 10:28
- Remove scala-2.12- version-specific source directories
- Update build.sbt to only support Scala 2.13 and 3.3.4
- Update CI workflow to remove 2.12 from test matrix
- Use Scala 2.13+ APIs (IterableOnce, jdk.CollectionConverters)

BREAKING: Scala 2.12 is no longer supported.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Move nested class definitions outside test methods for Scala 3 compatibility:
- ClosureCleanerAdditionalSpec: Move test helper classes to class level and
  companion object to avoid NoClassDefFoundError in Scala 3
- ExternalizerAdditionalSpec: Add explicit type annotations for heterogeneous
  collections to avoid type inference issues in Scala 3

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Use explicit type parameters Map[String, Any] and Externalizer[Map[String, Any]]
to avoid Scala 3 type inference issues while maintaining original test coverage.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@fangchenli fangchenli merged commit c9e5106 into develop Jan 30, 2026
11 checks passed
@fangchenli fangchenli deleted the drop-scala-2.12 branch January 30, 2026 19:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant